Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fixed localisation for notes block #943

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: fixed localisation for notes block #943

wants to merge 1 commit into from

Conversation

ekiuled
Copy link

@ekiuled ekiuled commented Dec 14, 2024

Description

Fixed localisation for notes block: pass file language to notes pluging instead of always using yfm args.

Related issue: #890

@martyanovandrey
Copy link
Contributor

Thank you for your contribution! Here are a few points that need to be addressed:

  1. I built our documentation with the changes from the current pull request, but I still encountered an error regarding the section titles being in Russian. Could you please let me know which documentation you tested on and with what parameters you ran the build? This information will help us identify the issue more effectively.

  2. To facilitate the validation of this PR, it would be helpful to include tests. Tests will ensure that everything is functioning as expected and make it easier for us to proceed with the integration.

  3. The generateStaticMarkup function is raising an error due to missing type definitions.

Once these issues have been addressed, we can proceed with further review. Thank you for your efforts!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants